feat(api): Probe for @ mentions in TUI append to prompt API#3996
feat(api): Probe for @ mentions in TUI append to prompt API#3996ewired wants to merge 2 commits intoanomalyco:devfrom
Conversation
|
The only failing test is failing on other recent PRs and pertains to the |
94e3b63 to
3882206
Compare
772b621 to
eb855e1
Compare
3882206 to
da34e9b
Compare
da34e9b to
c82ab3f
Compare
|
The new approach adds the ability to specify parts on TuiEvent.PromptAppend and must extract the mentions using the Instance information during the response to /tui/append-prompt. This means /tui/append-prompt has different capabilities than TuiEvent.PromptAppend and couldn't be a direct pass-through anymore. |
030d134 to
8770aaa
Compare
f1dc981 to
3e15a39
Compare
df8bdf9 to
0dd5039
Compare
e947abb to
07f200c
Compare
f8ee907 to
6a9856d
Compare
07f200c to
37d68d0
Compare
f28deb9 to
c8e214a
Compare
c8e214a to
f1596f5
Compare
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
00637c0 to
71e0ba2
Compare
f1ae801 to
08fa7f7
Compare
f026eb8 to
3fdb6c6
Compare
ca4d99d to
4e3274d
Compare
5e7a4b6 to
8578c54
Compare
ab8f9c1 to
a385fd6
Compare
|
I am retiring from this PR as the linked issue is now stale and there is an alternative way to achieve this by creating a session and posting messages to it. If you look at the network logs of the web UI you can create an equivalent solution, but you have to resolve the mentions to file URLs externally like the web UI does. The neglected |
1.0 removed the ability for POST /tui/append-prompt to include @ mentioned files and agents. This patch extracts and uses the mention resolution for files and agents used by command templates, marking the mentions as if they were added from Autocomplete. The behavior of the append prompt TUI command is also changed to move the cursor to the end before inserting, and I don't know if pre-1.0 versions did this. Unlike Autocomplete, it can resolve files relative to the home directory mentioned with
~/, which might be undesired.Fixes #2129